Skip to content

fix: add missing kosli evaluate input to CLI navigation#176

Merged
dangrondahl merged 1 commit intomainfrom
fix/175-evaluate-input-nav
Apr 23, 2026
Merged

fix: add missing kosli evaluate input to CLI navigation#176
dangrondahl merged 1 commit intomainfrom
fix/175-evaluate-input-nav

Conversation

@dangrondahl
Copy link
Copy Markdown
Contributor

@dangrondahl dangrondahl commented Apr 23, 2026

Summary

  • Adds client_reference/kosli_evaluate_input to the kosli evaluate navigation group in config/navigation.json
  • The page file already exists (generated from CLI v2.15.0) but was never added to navigation

Root cause: the update-cli-nav.py script silently fails because navigation moved from docs.json to config/navigation.json via $ref — tracked in #174.

Closes #175

The page exists in client_reference/ but was never added to navigation
because the update-cli-nav.py script silently fails (see #174).

Closes #175
@dangrondahl dangrondahl requested a review from a team as a code owner April 23, 2026 10:57
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 23, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
kosli 🟢 Ready View Preview Apr 23, 2026, 10:57 AM

@dangrondahl dangrondahl merged commit 46d9fad into main Apr 23, 2026
5 checks passed
@dangrondahl dangrondahl deleted the fix/175-evaluate-input-nav branch April 23, 2026 11:02
dangrondahl added a commit that referenced this pull request Apr 23, 2026
## Summary

- Updates `scripts/update-cli-nav.py` to read/write
`config/navigation.json` directly instead of `docs.json`
- Updates `.github/workflows/update-cli-docs.yml` to pass `--nav-file
config/navigation.json`
- Script now exits with error if CLI Reference section is not found
(previously failed silently)
- Also picks up the missing `kosli evaluate input` nav entry (same as
#176)

**Root cause:** When navigation was split from `docs.json` into
`config/navigation.json` via `$ref`, the script kept targeting
`docs.json` where it found `{"$ref": ...}` instead of the actual nav
structure.

Closes #174
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: add missing kosli evaluate input to CLI navigation

2 participants